[feat] 핀 목록 (좌표 필터링x) API 연동 #95

jihyunniiii · 2024-01-10T12:28:10Z

Related issue 🛠

closed [feat] 핀 목록 (좌표 필터링x) API 연동 #94

Work Description ✏️

핀 목록 (좌표 필터링 x) API를 연동하였습니다.
지도의 minZoom 을 설정하였습니다.

Screenshot 📸

Screen_recording_20240110_212511.mp4

Uncompleted Tasks 😅

N/A

To Reviewers 📢

더미 파일 보고 이해하기 힘들면 이 PR 보고 서버 통신 진행하세요 ~
서울, 독도, 제주도에 핀 하나씩 찍힘 ㅋㅋ

HAJIEUN02

1차 코리 ㅋㅋ. 추가할게염 고생핑

HAJIEUN02 · 2024-01-10T12:40:44Z

app/src/main/java/org/sopt/pingle/data/datasource/remote/MapRemoteDataSource.kt

+import org.sopt.pingle.util.base.BaseResponse
+
+interface MapRemoteDataSource {
+    suspend fun getPinListWithoutFiltering(teamId: Long, category: String?): BaseResponse<List<ResponsePinListDto>>


아 애초에 여기서부터 List로 선언을 해줘야 하는 군여,, Service에서만 선언하니까 오류가 난거였티비..~

HAJIEUN02 · 2024-01-10T12:45:03Z

app/src/main/java/org/sopt/pingle/data/datasourceimpl/remote/MapRemoteDataSourceImpl.kt

+import org.sopt.pingle.data.service.MapService
+import org.sopt.pingle.util.base.BaseResponse
+
+class MapRemoteDataSourceImpl @Inject constructor(


@Inject 어노테이션은 디펜던시에 힐트 설정 안 하면 못 쓰는 앤가요?

네 인젝션은 힐트에서 사용하는 어노테이션이기때문에 힐트를 사용할 때만 가능한 것으로 알고있습니다

HAJIEUN02 · 2024-01-10T12:49:52Z

app/src/main/java/org/sopt/pingle/data/datasourceimpl/remote/MapRemoteDataSourceImpl.kt

+        teamId: Long,
+        category: String?
+    ): BaseResponse<List<ResponsePinListDto>> =
+        mapService.getPinListWithoutFiltering(teamId = teamId, category = category)


데이터소스에서 함수 추상화하고 데이터소스Impl에서 서비스에 접근해서 통신한 값들 불러오는 함수를 호출하는 거까지 이해 완...
갑자기 궁금한게 생겼는데 서비스는 왜 인터페이스인걸가요? 얘네는 오버라이드 해야되는 함수를 구현하는 것도 아닌데.. 흠냐

구현해야할 코드가 없고 서버통신에서 필요한 부분이 어노테이션과 매개변수에 있어서 그런게 아닐까요..?!

HAJIEUN02

2차 코리 끝 ㅎ.ㅎ

HAJIEUN02 · 2024-01-10T12:52:06Z

app/src/main/java/org/sopt/pingle/data/service/MapService.kt

+        @Query(CATEGORY) category: String?
+    ): BaseResponse<List<ResponsePinListDto>>
+
+    companion object {


이건 URI에서 중복되는 부분들 일부러 다 상수화한건가요? 이렇게 해야 하는 이유는 무엇인가여? 몇 개 없을 때 상수화하는게 가독성에 조을까요? 그냥 나중에 규모가 커졌을 때를 대비해서 미리 상수화하는건가여

HAJIEUN02 · 2024-01-10T13:20:46Z

app/src/main/java/org/sopt/pingle/di/DataSourceModule.kt

+
+    @Binds
+    @Singleton
+    abstract fun bindsMapRemoteDataSource(mapRemoteDataSourceImpl: MapRemoteDataSourceImpl): MapRemoteDataSource


이 부분이 제일 이해가 안되는데................................. 힐트 부분인건가요? 대충 뭘 많이 모아둔.. 데이터소스를 모아둔 부분인 건 알겠는데요.........

HAJIEUN02 · 2024-01-10T13:23:33Z

app/src/main/java/org/sopt/pingle/di/ServiceModule.kt

@@ -16,4 +18,9 @@ object ServiceModule {
    @Singleton
    fun providesDummyService(@Pingle retrofit: Retrofit): DummyService =
        retrofit.create(DummyService::class.java)
+
+    @Provides


@BINDS와 @provides의 차이가 무엇인가욤?

혼자 공부완ㅋㅅㅋ

HAJIEUN02 · 2024-01-10T13:27:14Z

app/src/main/java/org/sopt/pingle/di/RepositoryModule.kt

 import org.sopt.pingle.domain.repository.DummyRepository
+import org.sopt.pingle.domain.repository.MapRepository

 @Module
 @InstallIn(SingletonComponent::class)
 abstract class RepositoryModule {
    @Binds
    @Singleton


@singleton 객체의 인스턴스가 1개만 생성되는 싱글톤 패턴을 구현하기 위해 존재하는 어노테이션인가요? 이 어노테이션의 의미가 궁금합니다링구

다은언니와 공부완
`~

HAJIEUN02 · 2024-01-10T13:28:59Z

app/src/main/java/org/sopt/pingle/domain/usecase/GetPinListWithoutFilteringUseCase.kt

+class GetPinListWithoutFilteringUseCase(
+    private val mapRepository: MapRepository
+) {
+    suspend operator fun invoke(teamId: Long, category: String?): Flow<List<PinEntity>> =


왜 함수 이름 선언 안 해주고 invoke를 쓰는 건가염? invoke의 이점이 무엇인지 궁금합니둥

HAJIEUN02 · 2024-01-10T13:30:17Z

app/src/main/java/org/sopt/pingle/presentation/ui/main/home/map/MapFragment.kt


+@AndroidEntryPoint


@androidentrypoint 는 프로젝트의 각 Android 클래스에 관한 개별 Hilt 구성요소를 생성합니다....... 대충 뇌빼고 따라 쳐야 하는 부분 그냥 규칙.. 머 그런거겠죠

HAJIEUN02 · 2024-01-10T13:40:12Z

app/src/main/java/org/sopt/pingle/presentation/ui/main/home/map/MapFragment.kt

-        mapViewModel.category.flowWithLifecycle(lifecycle).onEach {
-            // TODO 서버 통신 구현 시 삭제 예정
-            showToast(it?.name ?: "null")
+        mapViewModel.category.flowWithLifecycle(lifecycle).onEach { category ->


좋다.. 이 코드.. 쇽샥 가능하겠군요..

Doreminwoo · 2024-01-10T18:25:41Z

app/src/main/java/org/sopt/pingle/data/service/MapService.kt

+    companion object {
+        const val VERSION = "v1"
+        const val TEAMS = "teams"
+        const val PINS = "pins"
+        const val TEAM_ID = "teamId"
+        const val CATEGORY = "category"
+    }


이 부분 상수화를 모든 Service 파일마다 해야하는건가요!?

더 궁금한점이 나중에 url이 바뀔 가능성과 가독성을 위한거라면 상수화한 파일만 따로만들어서 서버통신 부분에서 공통되게 사용해야하는게 맞는 방법이라고 생각이 들어요!

Doreminwoo

많이 보고 배웠습니다요~

Doreminwoo · 2024-01-12T18:35:04Z

app/src/main/java/org/sopt/pingle/data/datasourceimpl/remote/MapRemoteDataSourceImpl.kt

+import org.sopt.pingle.data.service.MapService
+import org.sopt.pingle.util.base.BaseResponse
+
+class MapRemoteDataSourceImpl @Inject constructor(


네 인젝션은 힐트에서 사용하는 어노테이션이기때문에 힐트를 사용할 때만 가능한 것으로 알고있습니다

Doreminwoo · 2024-01-12T18:38:02Z

app/src/main/java/org/sopt/pingle/data/datasourceimpl/remote/MapRemoteDataSourceImpl.kt

+        teamId: Long,
+        category: String?
+    ): BaseResponse<List<ResponsePinListDto>> =
+        mapService.getPinListWithoutFiltering(teamId = teamId, category = category)


구현해야할 코드가 없고 서버통신에서 필요한 부분이 어노테이션과 매개변수에 있어서 그런게 아닐까요..?!

Doreminwoo · 2024-01-12T18:38:41Z

app/src/main/java/org/sopt/pingle/data/repository/MapRepositoryImpl.kt

+            ).data.map { pin ->
+                pin.toPinEntity()
+            }


이런식으로 하는거 몰랐었는데 코드짜면서 막힐 때 보고 배웠습니닷!

Doreminwoo · 2024-01-12T18:40:03Z

app/src/main/java/org/sopt/pingle/data/service/MapService.kt

+    companion object {
+        const val VERSION = "v1"
+        const val TEAMS = "teams"
+        const val PINS = "pins"
+        const val TEAM_ID = "teamId"
+        const val CATEGORY = "category"
+    }


더 궁금한점이 나중에 url이 바뀔 가능성과 가독성을 위한거라면 상수화한 파일만 따로만들어서 서버통신 부분에서 공통되게 사용해야하는게 맞는 방법이라고 생각이 들어요!

HAJIEUN02

우헤헹 멋찌다

jihyunniiii added 4 commits January 10, 2024 21:22

[feat] #94 핀 목록 (좌표 필터링 x) API 연동

907f284

[chore] #94 AuthInterceptor Authorization 수정

f5f4dcf

[chore] #94 pinMapper 포지션 수정

4958829

[feat] #94 핀 목록 (좌표 필터링 x) API 연동 후 로직 구현

16862e1

jihyunniiii added ✨ feat ✨ 🐸 지현 🐸 labels Jan 10, 2024

jihyunniiii added this to the 1차 스프린트 API 연동 milestone Jan 10, 2024

jihyunniiii self-assigned this Jan 10, 2024

jihyunniiii requested a review from a team as a code owner January 10, 2024 12:28

jihyunniiii added the 😵‍💫 Code Review wanted 😵‍💫 label Jan 10, 2024

HAJIEUN02 approved these changes Jan 10, 2024

View reviewed changes

jihyunniiii merged commit 8ba5827 into develop Jan 10, 2024
1 check passed

jihyunniiii deleted the feat-pin-list-api-connection branch January 10, 2024 12:50

HAJIEUN02 approved these changes Jan 10, 2024

View reviewed changes

Doreminwoo reviewed Jan 10, 2024

View reviewed changes

Doreminwoo reviewed Jan 12, 2024

View reviewed changes

HAJIEUN02 reviewed Jan 12, 2024

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[feat] 핀 목록 (좌표 필터링x) API 연동 #95

[feat] 핀 목록 (좌표 필터링x) API 연동 #95

jihyunniiii commented Jan 10, 2024 •

edited

Loading

HAJIEUN02 left a comment

HAJIEUN02 Jan 10, 2024

HAJIEUN02 Jan 10, 2024

Doreminwoo Jan 12, 2024

HAJIEUN02 Jan 10, 2024

Doreminwoo Jan 12, 2024

HAJIEUN02 left a comment

HAJIEUN02 Jan 10, 2024

HAJIEUN02 Jan 10, 2024

HAJIEUN02 Jan 10, 2024

HAJIEUN02 Jan 12, 2024

HAJIEUN02 Jan 10, 2024

HAJIEUN02 Jan 12, 2024

HAJIEUN02 Jan 10, 2024

HAJIEUN02 Jan 10, 2024

HAJIEUN02 Jan 10, 2024

Doreminwoo Jan 10, 2024

Doreminwoo Jan 12, 2024

Doreminwoo left a comment

Doreminwoo Jan 12, 2024

Doreminwoo Jan 12, 2024

Doreminwoo Jan 12, 2024

Doreminwoo Jan 12, 2024

HAJIEUN02 left a comment


		@AndroidEntryPoint

[feat] 핀 목록 (좌표 필터링x) API 연동 #95

[feat] 핀 목록 (좌표 필터링x) API 연동 #95

Conversation

jihyunniiii commented Jan 10, 2024 • edited Loading

Related issue 🛠

Work Description ✏️

Screenshot 📸

Uncompleted Tasks 😅

To Reviewers 📢

HAJIEUN02 left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

HAJIEUN02 left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Doreminwoo left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

HAJIEUN02 left a comment

Choose a reason for hiding this comment

jihyunniiii commented Jan 10, 2024 •

edited

Loading